Skip to content

Conversation

orta
Copy link
Contributor

@orta orta commented Jan 6, 2021

Starts to handle the feedback in #42067

@typescript-bot typescript-bot added Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Jan 6, 2021
@orta
Copy link
Contributor Author

orta commented Jan 6, 2021

@typescript-bot run dt

@typescript-bot
Copy link
Collaborator

typescript-bot commented Jan 6, 2021

Heya @orta, I've started to run the parallelized Definitely Typed test suite on this PR at bfa23c7. You can monitor the build here.

@orta orta added this to the TypeScript 4.2.0 milestone Jan 6, 2021
@orta
Copy link
Contributor Author

orta commented Jan 6, 2021

OK, so the biggest issue seems to be that

interface ImportMeta {
  url?: string
} 

conflicts with Nodes version which isn't nullable, I'll look into making the DOM version non-null.

@orta
Copy link
Contributor Author

orta commented Jan 6, 2021

@typescript-bot run dt

@typescript-bot
Copy link
Collaborator

typescript-bot commented Jan 6, 2021

Heya @orta, I've started to run the parallelized Definitely Typed test suite on this PR at 65d6e8a. You can monitor the build here.

@orta
Copy link
Contributor Author

orta commented Jan 6, 2021

OK, this is a big improvement - I'm going to merge this, so that it doesn't block others using the @typescript-bot run dt entirely

@typescript-bot
Copy link
Collaborator

typescript-bot commented Jan 6, 2021

Heya @orta, I've started to run the parallelized Definitely Typed test suite on this PR at 65d6e8a. You can monitor the build here.

@orta orta merged commit 35c8df0 into microsoft:master Jan 6, 2021
sandersn added a commit that referenced this pull request Jan 8, 2021
sandersn added a commit that referenced this pull request Jan 8, 2021
Zzzen pushed a commit to Zzzen/TypeScript that referenced this pull request Jan 16, 2021
* Revert the DOM query changes

* Adds baselines

* Hotfix for the removeal of the url nullabulity for testing
Zzzen pushed a commit to Zzzen/TypeScript that referenced this pull request Jan 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants